Skip to content

Update routing diagrams to reflect new architecture #977

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Aug 18, 2025

Conversation

shaun-nx
Copy link
Contributor

Proposed changes

This change updates two diagrams for NGINX Gateway Fabric to reflect architecture changes in v2.0.0

Closes nginx/nginx-gateway-fabric#3665

Checklist

Before sharing this pull request, I completed the following checklist:

Footnotes

  1. Potentially sensitive information includes personally identify information (PII), authentication credentials, and live URLs. Refer to the style guide for guidance about placeholder content.

@shaun-nx shaun-nx requested a review from a team as a code owner August 18, 2025 13:22
@github-actions github-actions bot added the documentation Improvements or additions to documentation label Aug 18, 2025
Copy link

Deploy Preview will be available once build job completes!

Name Link
😎 Deploy Preview https://frontdoor-test-docs.nginx.com/previews/docs/977/

@shaun-nx shaun-nx added the product/ngf Issues related to NGINX Gateway Fabric label Aug 18, 2025
@shaun-nx shaun-nx merged commit 6fa64d5 into main Aug 18, 2025
10 checks passed
@shaun-nx shaun-nx deleted the ngf/routing-diagrams branch August 18, 2025 13:35
@github-project-automation github-project-automation bot moved this from 🆕 New to ✅ Done in NGINX Gateway Fabric Aug 18, 2025
@sjberman
Copy link
Contributor

There needs to be a followup PR to actually update the images. This updated the mermaid that built them, but the docs import the PNG files, which weren't updated.

@shaun-nx
Copy link
Contributor Author

I'll put up that PR too. Thanks for catching that @sjberman

@shaun-nx
Copy link
Contributor Author

@sjberman made these updates here: #987
That PR also made it so that all of our mermaid diagram code is embedded in the markdown file too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation product/ngf Issues related to NGINX Gateway Fabric
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Update more docs for new architecture
4 participants